Fix phasar research tool to enable container build #664
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## vara-dev #664 +/- ##
============================================
- Coverage 68.20% 68.20% -0.01%
============================================
Files 333 333
Lines 25680 25682 +2
============================================
+ Hits 17516 17517 +1
- Misses 8164 8165 +1
☔ View full report in Codecov by Sentry. |
|
Should we merge this after the phasar changes were worked in? |
|
@fabianbs96 @boehmseb What's the current state of the phasar changes? Can we start to integrate this branch? |
|
The only remaining issue is to decide how to handle the llvm install. The options are:
|
|
If we can easily switch to 14 it's probably the simplest thing to do, especially as our LLVM state is also 14. |
We use llvm-14 in the container images anyway and for non-container builds the user should be responsible for supplying the correct llvm version.
|
@fabianbs96 @boehmseb What's the current state of this PR? We should resolve the llvm-14 issue to merge this one otherwise the branch will drift more and and more. |
This PR enables Phasar to be built in a container.
@MMory @pdschubert: There are two things that should be changed in phasar's
install-llvm.shscript for this to work properly:sudofor installation, this is not necessary when installing to a user-writeable directory (as we use here)